Skip to content

[Application] persist pressure result artifacts#268

Merged
learncold merged 4 commits into
mainfrom
codex/pressure-result-persistence
May 24, 2026
Merged

[Application] persist pressure result artifacts#268
learncold merged 4 commits into
mainfrom
codex/pressure-result-persistence

Conversation

@SilverSupplier
Copy link
Copy Markdown
Collaborator

@SilverSupplier SilverSupplier commented May 23, 2026

Summary

  • Persists pressure result summaries in project workspace result artifacts.
  • Persists pressure-specific risk snapshot details, including hotspots, exposed agents, and critical pressure events.
  • Adds persistence and codec coverage for pressure round-trip behavior and numeric legacy agent IDs.

Related Issue

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug (not rerun; existing configured build tree reused)
  • cmake --build --preset build-debug
  • ctest --preset test-debug
  • cmake --build --preset build-no-app-debug
  • ctest --preset test-no-app-debug
  • Not run (reason below)

Risks / Follow-up

SilverSupplier and others added 2 commits May 24, 2026 05:50
…-persistence

# Conflicts:
#	src/application/ResultArtifactsCodec.cpp
#	tests/ProjectPersistenceTests.cpp
@learncold learncold merged commit 5a569cd into main May 24, 2026
2 checks passed
@learncold learncold deleted the codex/pressure-result-persistence branch May 24, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants